Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Broken link in developer.md fixed #501

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

andoriyaprashant
Copy link

What this PR does / why we need it:

This change updates the broken link in the "Test your changes" section of the developer.md file. The original link was leading to a 404 error page. The updated link directs users to the correct page in the LitmusChaos documentation for running their first workflow.

Fixes #463

Special notes for your reviewer:

Checklist:

  • Fixes Broken link in documentation: Test the changes in developer.md #463
  • Labelled this PR & related issue with documentation tag
  • PR messages has document related information
  • Labelled this PR & related issue with breaking-changes tag
  • PR messages has breaking changes related information
  • Labelled this PR & related issue with requires-upgrade tag
  • PR messages has upgrade related information
  • Commit has unit tests
  • Commit has integration tests

@andoriyaprashant
Copy link
Author

Hello @Nageshbansal @neelanjan00 Sir can you please review this pull request and let me know if any changes are needed
thank you

@@ -157,7 +157,7 @@ Check your linting.
kubectl apply -f ./deploy/rbac.yaml
kubectl apply -f ./deploy/operator.yaml
```
- Run the chaos by following the [Litmus Docs](https://docs.litmuschaos.io/docs/getstarted/#install-chaos-experiments)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

andoriyaprashant and others added 5 commits September 14, 2024 17:09
Signed-off-by: andoriyaprashant <[email protected]>
Signed-off-by: dusdjhyeon <[email protected]>
Signed-off-by: andoriyaprashant <[email protected]>
Signed-off-by: Shubham Chaudhary <[email protected]>
Signed-off-by: andoriyaprashant <[email protected]>
Signed-off-by: andoriyaprashant <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken link in documentation: Test the changes in developer.md
3 participants